Skip to content

Add tier colour to ranked play ranking on profile overlay#38197

Open
diquoks wants to merge 1 commit into
ppy:masterfrom
diquoks:feature/ranked-colour
Open

Add tier colour to ranked play ranking on profile overlay#38197
diquoks wants to merge 1 commit into
ppy:masterfrom
diquoks:feature/ranked-colour

Conversation

@diquoks

@diquoks diquoks commented Jun 30, 2026

Copy link
Copy Markdown
Contributor
actual profile test scene
image image

@diquoks diquoks left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some side notes

Comment on lines +111 to 125
APIUserMatchmakingStatistics[] allStats = User.Value.User.MatchmakingStatistics;

if (stats.Length == 0)
if (allStats.Length == 0)
{
Hide();
return;
}

int? highestRank = null;
APIUserMatchmakingStatistics? highestRankStats = null;

foreach (var stat in stats)
foreach (var stats in allStats)
{
if (stat.Pool.Active && stat.Rank != null)
{
if (highestRank == null || stat.Rank < highestRank)
highestRank = stat.Rank;
}
if (stats.Pool.Active && (highestRankStats == null || stats.Rank < highestRankStats.Rank))
highestRankStats = stats;
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see reference

Comment on lines +20 to +23
public int Rank { get; set; }

[JsonProperty("rank_percent")]
public float RankPercent { get; set; }

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see reference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant